Skip to content

Log Integrations: Add SentrySwiftyBeaver Integration#7021

Merged
denrase merged 53 commits intomainfrom
feat/add-swifty-beaver-integration
Jan 20, 2026
Merged

Log Integrations: Add SentrySwiftyBeaver Integration#7021
denrase merged 53 commits intomainfrom
feat/add-swifty-beaver-integration

Conversation

@denrase
Copy link
Collaborator

@denrase denrase commented Dec 10, 2025 •
edited
Loading

#skip-changelog

📜 Description

  • Adds the SentrySwiftyBeaver integration to the 3rd-party-integrations/SentrySwiftyBeaver folder
  • Skipping changelog as this is not available to users through this repo

💡 Motivation and Context

Closes #6277
Relates to #6902

💚 How did you test it?

  • Runs pure swift tests against the current Package.swift version
  • This loads pre-build artefacts, so we always test against already released versions.

📝 Checklist

You have to check all boxes before merging:

  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.

@denrase denrase mentioned this pull request Dec 10, 2025
Closed
7 tasks
@denrase denrase added the ready-to-merge Use this label to trigger all PR workflows label Dec 10, 2025
@denrase denrase marked this pull request as ready for review December 10, 2025 17:45
@codecov
Copy link

codecov bot commented Dec 10, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.115%. Comparing base (d8db577) to head (d6fadb7).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##              main     #7021       +/-   ##
=============================================
+ Coverage   85.062%   85.115%   +0.052%     
=============================================
  Files          468       468               
  Lines        28305     28305               
  Branches     12532     12542       +10     
=============================================
+ Hits         24077     24092       +15     
+ Misses        4183      4172       -11     
+ Partials        45        41        -4

see 6 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d8db577...d6fadb7. Read the comment docs.

philprime
philprime approved these changes Dec 11, 2025
View reviewed changes
Copy link
Member

@philprime philprime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

denrase and others added 8 commits January 12, 2026 16:58
@denrase
Merge branch 'main' into feat/add-swift-log-integration
f28ac5b
@denrase
Changed log attribute keys from swift-log.* to code.* for better …
b54e2bb 
…clarity and consistency. Updated corresponding tests to reflect these changes.
@denrase
Merge branch 'main' into feat/add-swift-log-integration
828a2b7
@denrase @philprime
Update 3rd-party-integrations/SentrySwiftLog/README.md
ba3969b 
Co-authored-by: Philip Niedertscheider <phil.niedertscheider@sentry.io>
@denrase
Merge branch 'main' into feat/add-swift-log-integration
0f41df4
@denrase
Update codecov configuration and AGENTS.md to include 3rd-party integ…
cd37fe7 
…ration tests in coverage and unit test filters.
@denrase
Merge branch 'feat/add-swift-log-integration' of github.com:getsentry…
bb53032 
…/sentry-cocoa into feat/add-swift-log-integration
@denrase
Merge branch 'feat/add-swift-log-integration' into feat/add-swifty-be…
3a367a3 
…aver-integration

# Conflicts:
#	Utils/VersionBump/main.swift
@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2026 •
edited
Loading

Semver Impact of This PR

⚪ None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

This PR will not appear in the changelog.

🤖 This preview updates automatically when you update the PR.

cursor[bot]
cursor bot reviewed Jan 13, 2026
View reviewed changes
Base automatically changed from feat/add-swift-log-integration to main January 19, 2026 12:38
cursor[bot]
cursor bot reviewed Jan 20, 2026
View reviewed changes
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@denrase denrase enabled auto-merge (squash) January 20, 2026 10:27
@denrase denrase merged commit a424cf3 into main Jan 20, 2026
157 of 163 checks passed
@denrase denrase deleted the feat/add-swifty-beaver-integration branch January 20, 2026 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cursor cursor[bot] cursor[bot] left review comments

@philprime philprime philprime approved these changes

@philipphofmann philipphofmann Awaiting requested review from philipphofmann philipphofmann is a code owner

@noahsmartin noahsmartin Awaiting requested review from noahsmartin noahsmartin is a code owner

@itaybre itaybre Awaiting requested review from itaybre itaybre is a code owner

Assignees

No one assigned

Labels

ready-to-merge Use this label to trigger all PR workflows

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Log Integration: SwiftyBeaver

3 participants

@denrase @itaybre @philprime